Skip to content

feat(abstract-utxo)!: enable legacy tx format for mainnet#8363

Merged
OttoAllmendinger merged 2 commits intomasterfrom
BTC-0.enable-legacy-tx-on-mainnet
Mar 27, 2026
Merged

feat(abstract-utxo)!: enable legacy tx format for mainnet#8363
OttoAllmendinger merged 2 commits intomasterfrom
BTC-0.enable-legacy-tx-on-mainnet

Conversation

@OttoAllmendinger
Copy link
Copy Markdown
Contributor

Changes the supportedTxFormats property from readonly to mutable
and sets legacy format support based on network type. Legacy
format is now enabled for mainnet networks while remaining
disabled for testnet.

Adds a protected isMainnet() method to AbstractUtxoCoin class
that delegates to isMainnetCoin() helper and can be overridden
by subclasses if needed.

Issue: BTC-0

@OttoAllmendinger OttoAllmendinger force-pushed the BTC-0.enable-legacy-tx-on-mainnet branch from 07082c3 to 4b71978 Compare March 27, 2026 10:19
Add protected `isMainnet()` method to AbstractUtxoCoin class.
Default implementation delegates to `isMainnetCoin()` helper and
can be overridden by subclasses if needed.

Issue: BTC-0

Co-authored-by: llm-git <llm-git@ttll.de>
@OttoAllmendinger OttoAllmendinger force-pushed the BTC-0.enable-legacy-tx-on-mainnet branch 3 times, most recently from 8edd819 to 41fa398 Compare March 27, 2026 12:37
Changes the supportedTxFormats property from readonly to mutable
and sets legacy format support based on network type. Legacy
format is now enabled for mainnet networks while remaining
disabled for testnet.

Issue: BTC-0

Co-authored-by: llm-git <llm-git@ttll.de>
@OttoAllmendinger OttoAllmendinger force-pushed the BTC-0.enable-legacy-tx-on-mainnet branch from 41fa398 to ed26128 Compare March 27, 2026 12:43
@OttoAllmendinger OttoAllmendinger marked this pull request as ready for review March 27, 2026 13:05
@OttoAllmendinger OttoAllmendinger requested a review from a team as a code owner March 27, 2026 13:05
@OttoAllmendinger OttoAllmendinger merged commit 7ac007d into master Mar 27, 2026
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants